Skip to content

Remove non-backpack-specific commands #211

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
Copilot wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Remove non-backpack-specific commands #211

Copilot wants to merge 2 commits into from

Conversation

Copy link

Copilot AI commented Dec 18, 2025 •
edited
Loading

WHY

BEFORE - What was wrong? What was happening before this PR?

Package contained three generic commands (backpack:model, backpack:request, backpack:view) that duplicate Laravel's native make commands. These provide no backpack-specific functionality.

AFTER - What is happening after this PR?

Only backpack-specific commands remain. Users rely on Laravel's native commands for generic model/request/view generation. Backpack CRUD commands (backpack:crud-model, backpack:crud-request) continue working as they add CrudTrait and other backpack-specific features.

HOW

How did you achieve that, in technical terms?

Removed:

  • 3 command classes: ModelBackpackCommand, RequestBackpackCommand, ViewBackpackCommand
  • 5 stub files: model.stub, model-softdelete.stub, request.stub, view.stub, view-plain.stub
  • Command registrations and imports from GeneratorsServiceProvider
  • Documentation from README

Total: 522 lines deleted across 10 files.

Is it a breaking change or non-breaking change?

Breaking change. Projects using these commands must migrate to Laravel's native equivalents:

  • backpack:model → php artisan make:model
  • backpack:request → php artisan make:request
  • backpack:view → php artisan make:view

How can we test the before & after?

Before: Running php artisan list showed backpack:model, backpack:request, backpack:view

After: These commands are absent. Backpack-specific commands (backpack:crud, backpack:crud-model, backpack:crud-controller, etc.) remain available and functional.

Original prompt

This section details on the original issue you should resolve

<issue_title>[Proposal] Remove commands that are not backpack-specific</issue_title>
<issue_description>Currently we have:

  // backpack-specific
  backpack:chart                Create a ChartController and route
  backpack:chart-controller     Generate a Backpack ChartController
  backpack:config               Generate a backpack templated config
  backpack:crud                 Create a CRUD interface: Controller, Model, Request
  backpack:crud-controller      Generate a Backpack CRUD controller
  backpack:crud-model           Generate a Backpack CRUD model
  backpack:crud-operation       Generate a Backpack CRUD operation trait
  backpack:crud-request         Generate a Backpack CRUD request

  // not backpack specific
  backpack:model                Generate a backpack templated model
  backpack:request              Generate a backpack templated request
  backpack:view                 Generate a backpack templated view

I don't see a reason why we would keep the last 3. I'm pretty sure nobody uses them anyway, since Laravel has its own make commands for them nowadays.

I think it's about time we remove those commands and their stubs.</issue_description>

<agent_instructions>Be as surgical as possible</agent_instructions>

Comments on the Issue (you are @copilot in this section)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@tabacitu
Remove non-backpack-specific commands (model, request, view)
f9a93a0 
Co-authored-by: tabacitu <1032474+tabacitu@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove commands that are not backpack-specific Remove non-backpack-specific commands Dec 18, 2025
Copilot AI requested a review from tabacitu December 18, 2025 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tabacitu tabacitu Awaiting requested review from tabacitu

Assignees

@tabacitu tabacitu

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Proposal] Remove commands that are not backpack-specific

2 participants

@tabacitu